-
Notifications
You must be signed in to change notification settings - Fork 9
Making hypercube_url overrideable. Using variable for syn token. #9
Making hypercube_url overrideable. Using variable for syn token. #9
Conversation
I'm happy to close #13 over getting this merged. Basically ran into this issue earlier this morning. |
- Addresses #24 - Depends on: - islandora-deprecated/ansible-role-alpaca#13 - or - islandora-deprecated/ansible-role-alpaca#9
This is good to go. I took a glance at it and thought I still needed to add a default, but looks like I already did 😎 |
Groveling for a merge here @Islandora-Devops/committers |
would you like a full run build test? |
I'm spinning up an ubuntu right now. |
I'm happy to argue I've tested this a fair bit with our staging adaptation of the playbooks, and it's good to merge 😄 |
Granted, if my network is as bad today as yesterday, it will be more than an hour before I actually get to test it. 😅 |
Ubuntu checked out. Spinning up centos and then we can put this to bed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
GitHub Issue: Islandora/documentation#1409
What does this Pull Request do?
Makes a variable for hypercube's url and uses it. Also uses the pre-existing
islandora_syn_token
for configuring alpaca's http client. With this, we can eliminate having to updatekaraf.yml
in our inventory for provisioning remote servers.How should this be tested?
Pull in this PR, delete
roles/external
, thenvagrant up
. Everything should provision normally. Try creating an OCR derivative and see if it still works.Interested parties
@Islandora-Devops/committers